[monitor] Update @azure/opentelemetry-instrumentation-azure-sdk dependencies to ^1.0.0#38420
Conversation
…1.0.0 Agent-Logs-Url: https://github.com/Azure/azure-sdk-for-js/sessions/8837b4a7-670b-403c-81eb-c86f7c42b55c Co-authored-by: maorleger <753570+maorleger@users.noreply.github.com>
maorleger
requested review from
a team,
JacksonWeber,
MSNev,
bobogogo1990,
dargilco,
ganeshyb,
glharper,
hectorhdzg,
jeremydvoss,
jeremymeng,
lzchen and
rads-1996
as code owners
There was a problem hiding this comment.
Pull request overview
Updates dependent packages to use the stable @azure/opentelemetry-instrumentation-azure-sdk@^1.0.0 now that 1.0.0 is released, and refreshes the pnpm lockfile accordingly.
Changes:
- Bump
@azure/opentelemetry-instrumentation-azure-sdkdependency ranges from beta pins (1.0.0-beta.*) to^1.0.0in impacted packages. - Apply the same version range update in the corresponding samples’
package.jsonfiles. - Update
pnpm-lock.yamlto reflect the new specifiers and remove no-longer-needed beta snapshot entries.
Reviewed changes
Copilot reviewed 8 out of 9 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| sdk/monitor/monitor-opentelemetry/package.json | Updates runtime dependency to @azure/opentelemetry-instrumentation-azure-sdk@^1.0.0. |
| sdk/ai/ai-projects/package.json | Updates dev dependency pin to ^1.0.0. |
| sdk/ai/ai-inference-rest/package.json | Updates dev dependency pin to ^1.0.0. |
| sdk/ai/ai-inference-rest/samples/v1-beta/typescript/package.json | Updates sample dependency pin to ^1.0.0. |
| sdk/ai/ai-inference-rest/samples/v1-beta/javascript/package.json | Updates sample dependency pin to ^1.0.0. |
| sdk/ai/ai-agents/package.json | Updates dev dependency pin to ^1.0.0. |
| sdk/ai/ai-agents/samples/v1-beta/typescript/package.json | Updates sample dependency pin to ^1.0.0. |
| sdk/ai/ai-agents/samples/v1-beta/javascript/package.json | Updates sample dependency pin to ^1.0.0. |
| pnpm-lock.yaml | Lockfile refresh reflecting updated specifiers and removing old beta snapshots. |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
…ion-azure-sdk@1.0.0 Agent-Logs-Url: https://github.com/Azure/azure-sdk-for-js/sessions/1bd6d034-7c5b-4fe0-b7a3-72c7dd3c4739 Co-authored-by: maorleger <753570+maorleger@users.noreply.github.com>
|
@copilot resolve the merge conflicts in this pull request |
…resolve conflicts Co-authored-by: maorleger <753570+maorleger@users.noreply.github.com>
Resolved the merge conflicts in commit |
|
@copilot resolve the merge conflicts in this pull request |
…resolve conflicts Co-authored-by: maorleger <753570+maorleger@users.noreply.github.com>
Resolved the merge conflicts in commit |
|
Hi @copilot. We've noticed that new changes have been pushed to this pull request. Because it is set to automatically merge, we've reset the approvals to allow the opportunity to review the updates. |
github-actions
Bot
dismissed stale reviews from jeremymeng and JacksonWeber
Hi @jeremymeng. We've noticed that new changes have been pushed to this pull request. Because it is set to automatically merge, we've reset the approvals to allow the opportunity to review the updates.
Now that
@azure/opentelemetry-instrumentation-azure-sdk1.0.0 is released, update all dependent packages from beta version pins to^1.0.0.Packages impacted by this PR
@azure/monitor-opentelemetry@azure/ai-agents@azure/ai-inference(rest)@azure/ai-projectsIssues associated with this PR
Describe the problem that is addressed by this PR
Dependent packages were pinned to
^1.0.0-beta.9or exact1.0.0-beta.7versions of@azure/opentelemetry-instrumentation-azure-sdk. With 1.0.0 now published, these should resolve to the stable release. Additionally, the perf test version matrix insdk/monitor/monitor-opentelemetry/perf-tests.ymlwas also pinned to1.0.0-beta.5and has been updated to1.0.0.What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Straightforward version range update to
^1.0.0across all dependents +pnpm installto update the lockfile. The perf test config version matrix entry was updated to the stable1.0.0pin to match. Merge conflicts withmainwere resolved across multiple rounds by retaining our^1.0.0version bump for@azure/opentelemetry-instrumentation-azure-sdkalongside new changes introduced inmain— including the@modelcontextprotocol/sdkdevDependency insdk/ai/ai-projects, and the Monitor Distro 1.17.0 / Exporter beta.40 release bumps (@azure/monitor-opentelemetry-exporter@1.0.0-beta.40,@azure/monitor-opentelemetry@^1.17.0). The lockfile was regenerated withpnpm installafter each merge.Are there test cases added in this PR? (If not, why?)
No — this is a dependency version bump with no behavioral changes.
Provide a list of related PRs (if any)
N/A
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists